PR de correcao #2
Conversation
| # Getting Started with Create React App | ||
|
|
||
| This project was bootstrapped with [Create React App](https://github.com/facebook/create-react-app). | ||
|
|
||
| ## Available Scripts | ||
|
|
||
| In the project directory, you can run: | ||
|
|
||
| ### `npm start` | ||
|
|
||
| Runs the app in the development mode.\ | ||
| Open [http://localhost:3000](http://localhost:3000) to view it in the browser. | ||
|
|
||
| The page will reload if you make edits.\ | ||
| You will also see any lint errors in the console. |
There was a problem hiding this comment.
Esse README deveria ter sido editado com as informacoes da aplicacao: fotos, link do surge e outras observacoes sobre o projeto.
| if(this.state.valorInputNomeUsuario !== '' && this.state.valorInputMensagemUsuario !== ''){ | ||
| const mensagensAtualizadas = [...this.state.mensagens, novaMensagem]; | ||
|
|
||
| this.setState({ mensagens: mensagensAtualizadas }); | ||
| this.state.valorInputNomeUsuario = '' | ||
| this.state.valorInputMensagemUsuario = '' | ||
| } else { | ||
| alert('Preencha seu nome e mensagem desejada') | ||
| } |
| const ContainerMensagem = Styled.div` | ||
| width: 100vw; | ||
| background-color: #000; | ||
| display: flex; | ||
| flex-direction: column; | ||
| justify-content: ${(props) => { | ||
| if (props.cor === "eu") { | ||
| return "center" | ||
| } else { | ||
| return "flex-end" | ||
| } | ||
| }} |
There was a problem hiding this comment.
Parabens pela logica com props e styled-components, ta muito boa! So cuidado com a indentacao!
| onClick={() => this.removerMensagem(mensagem.nomeUsuario)}> */ | ||
|
|
||
| /* const ContainerTitulo = styled.div` | ||
| display: flex; | ||
| height: ${(props) => { | ||
| if (props.tamanho === "grande") { | ||
| return "300px"; | ||
| } else { | ||
| return "200px"; | ||
| } | ||
| }}; | ||
| background-color: ${(props) => props.cor}; | ||
| align-items: center; | ||
| justify-content: center; | ||
| width: 300px; | ||
| margin-bottom: 30px; | ||
| `; */ | ||
|
|
||
|
|
||
| /* | ||
| import React from 'react' | ||
| import Styled from 'styled-components' | ||
| import App from './App.js'; | ||
|
|
||
|
|
||
| const NomeU = Styled.div` | ||
| height: 0.5vh; | ||
| margin: 10px 10px 0; | ||
| border-radius: 0 7px 0 0; | ||
| padding: 10px; | ||
| width: 20vw; | ||
| background-color: ${(props) => { | ||
| if (props.cor === "eu") { | ||
| return "#DBF6C5" | ||
| } else { |
There was a problem hiding this comment.
Codigo comentado pode e deve ser retirado antes da entrega :)
|
Oi, grupo! O projeto de voces foi avaliado como: acima do esperado! Parabens! 👏 Todas as aplicações obrigatórias e nao obrigatórias funcionam bem e a logica usada foi muito acertada! Queria apenas pedir mais atencao antes da entrega do projeto: o README esta como veio por padrao com a criacao do app React, mas deveria conter as informacoes importantes relativas ao projeto de voces. Percebi tambem alguns probleminhas de identacao, lembrem de revisar o código antes de enviar! |
ghost
requested a review
from 2 participants
Oie! Esse e um PR de correção que nao precisa ser mergeado. A avaliação do projeto esta mais abaixo!